Skip to content

BlackPerl DFIR Academy #5

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
May 30, 2023
Merged

Conversation

archanchoudhury
Copy link
Contributor

Hello Florian, I am raising this PR to add BlackPerl DFIR Academy to be included in the repo. For the preview of the courses, our YouTube can be visited or each course in the academy comes up with some trail topics which can be viewed after a Free Sign Up.

@Neo23x0
Copy link
Contributor

Neo23x0 commented May 26, 2023

Hi Archan,

thanks for the submission.
Could you please list all 5/6 courses that you offer separately in the YAML file. You could check the YAML files of the other training providers for guidance.
You have 6 trainings. It should take you a lot of your time.

Why did you decide against using a currency that most people know? (like e.g. USD)
I tried to select the price to copy it and paste it into a google search, but I cannot select it.
So I, as a user, have no idea what this currency is and how it converts to a currrently that I know.
Screenshot 2023-05-26 at 11 20 47

@archanchoudhury
Copy link
Contributor Author

Hello Florian,

Thanks for reverting back. Yes, I will convert the YAML file and make the adjustments. Actually we are coming up with more courses and the plan is to publish 1 course / month in this year. So do you want me to keep it editing?

For the currency, since we initially focused on India based users, and hence the platform payment gateway is configured to show INR as currency. However, when you try to buy a course outside of India, the payment gateway will convert that for you. Please see the screen shot below for USD:
image

Similarly people can choose the more option above as well to select their currency as they need.

Copy link

@cany1985 cany1985 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tracy0880550287
Copy link

Thanks for helping us maintain the quality of our codebase

@Neo23x0 Neo23x0 merged commit 3810c04 into Cyber-Security-Hub:main May 30, 2023
@Neo23x0
Copy link
Contributor

Neo23x0 commented May 30, 2023

@archanchoudhury : I just noticed

  1. that you just gave me one training although you offer six

Screenshot 2023-05-30 at 17 26 35

Screenshot 2023-05-30 at 17 26 39

and 2. that I cannot copy & paste from your website, which is super annoying.

I would've updated the YAML myself, but since I would have to type off every training title and description, I'm just removing you entry as long as the data is incomplete.

@Neo23x0
Copy link
Contributor

Neo23x0 commented May 30, 2023

You can work on the changes here:
#8

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants